Skip to content

chore: remove arc chart enableV1Beta1APIs flag#1262

Merged
weng271190436 merged 2 commits intoAzure:mainfrom
weng271190436:weiweng/remove-arc-v1beta1-flag
Mar 5, 2026
Merged

chore: remove arc chart enableV1Beta1APIs flag#1262
weng271190436 merged 2 commits intoAzure:mainfrom
weng271190436:weiweng/remove-arc-v1beta1-flag

Conversation

@weng271190436
Copy link
Member

@weng271190436 weng271190436 commented Mar 2, 2026

Description of your changes

Remove arc chart enableV1Beta1APIs flag. We are planning to use v1beta1 forever and it is enabled by default

Corresponding CCP changes:
https://dev.azure.com/msazure/CloudNativeCompute/_git/aks-rp/pullrequest/14889548
https://dev.azure.com/msazure/CloudNativeCompute/_git/aks-rp/pullrequest/14919559

Fixes #

I have:

  • Run make reviewable to ensure this PR is ready for review.

How has this code been tested

Special notes for your reviewer

Signed-off-by: Wei Weng <Wei.Weng@microsoft.com>
@MuhammadAliFleet
Copy link
Contributor

Is this being removed because the default value is now set to true?

enableV1Alpha1APIs = flag.Bool("enable-v1alpha1-apis", false, "If set, the agents will watch for the v1alpha1 APIs. This is deprecated and will be removed soon.")

@weng271190436
Copy link
Member Author

Is this being removed because the default value is now set to true?

enableV1Alpha1APIs = flag.Bool("enable-v1alpha1-apis", false, "If set, the agents will watch for the v1alpha1 APIs. This is deprecated and will be removed soon.")

yes. We are planning to enable v1beta1 all the time without needing a flag.

Also I am making corresponding changes on CCP chart
https://dev.azure.com/msazure/CloudNativeCompute/_git/aks-rp/pullrequest/14889548
https://dev.azure.com/msazure/CloudNativeCompute/_git/aks-rp/pullrequest/14919559

@weng271190436 weng271190436 merged commit 27694c5 into Azure:main Mar 5, 2026
45 of 51 checks passed
@weng271190436 weng271190436 deleted the weiweng/remove-arc-v1beta1-flag branch March 5, 2026 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants